Fix: correct speechText for Message_Stream_Event #28
Merged
sebvanleuven merged 11 commits intomainfrom Jan 28, 2026
Merged
Conversation
There was a problem hiding this comment.
2 issues found across 7 files
Prompt for AI agents (all issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<file name="README.md">
<violation number="1" location="README.md:135">
P2: The example now references `Message` but the import was removed. This makes the snippet invalid when copied. Add `Message` to the import list.</violation>
</file>
<file name="examples/persona_interactive_video.py">
<violation number="1" location="examples/persona_interactive_video.py:41">
P3: The comment says the log level was reduced to WARNING, but the code now sets INFO. This mismatch makes the logging intent unclear for maintainers.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
There was a problem hiding this comment.
2 issues found across 7 files
Prompt for AI agents (all issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<file name="src/anam/_streaming.py">
<violation number="1" location="src/anam/_streaming.py:345">
P2: Reset _data_channel_open to False when creating a new data channel; otherwise stale True values from a previous session can cause sends while the new channel is still connecting.</violation>
</file>
<file name="examples/persona_interactive_video.py">
<violation number="1" location="examples/persona_interactive_video.py:41">
P3: The comment on the logging level no longer matches the code (INFO vs “reduced to WARNING”), which is misleading for maintenance.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixing message type for Message_Stream_Event. Fixing message processing, fixing message history
Expand interactive example using MESSAGE_STREAM_EVENTS for live caption and printing the whole message history at the end of a session.
Summary by cubic
Adds real-time message streaming support using speechText events and builds automatic conversation history. Fixes message parsing and improves the interactive example with live captions and transcript printing.
New Features
Bug Fixes
Written for commit 8229024. Summary will update on new commits.